Skip to content

MLE-24531 Debugging failing DMSDK tests #976

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 10 commits into from
Closed

MLE-24531 Debugging failing DMSDK tests #976

wants to merge 10 commits into from

Conversation

@rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Oct 7, 2025

Also going to move docker-compose to the root dir once this PR is ready.

@github-actions
Copy link

github-actions bot commented Oct 7, 2025 •
edited
Loading

Copyright Validation Results
Total: 11 | Passed: 0 | Failed: 5 | Skipped: 6 | at: 2025-10-09 11:22:11 UTC | commit: be84796

❌ Failed Files

  • test-basic/endpoint-caller.js

    Error:

    - Copyright format does not match expected format

    Expected header:

    Copyright (c) 2015-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.

  • test-basic/lockForUpdate-test.js

    Error:

    - Copyright format does not match expected format

    Expected header:

    Copyright (c) 2015-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.

  • test-basic/plan-builder-generated.js

    Error:

    - Copyright format does not match expected format

    Expected header:

    Copyright (c) 2015-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.

  • test-basic/service-caller.js

    Error:

    - Copyright format does not match expected format

    Expected header:

    Copyright (c) 2015-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.

  • test-basic/validateDoc-test.js

    Error:

    - Copyright format does not match expected format

    Expected header:

    Copyright (c) 2015-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.

⏭️ Skipped (Excluded) Files

  • .copyrightconfig
  • .env
  • .gitignore
  • CONTRIBUTING.md
  • Jenkinsfile
  • docker-compose.yaml

🛠️ Guidance

Follow these steps to fix the failed files:

  1. Insert the expected header at the very top (within first 20 lines) of each failed file.
  2. Ensure the year range matches the configuration (start year through current year).
  3. Do not alter spacing or punctuation in the header line.
  4. Commit and push the changes to update this check.

@rjrudin rjrudin force-pushed the feature/fix-tests branch 3 times, most recently from 0af8e7c to 36f9a4e Compare October 8, 2025 17:18
rjrudin added 3 commits October 8, 2025 15:01
@rjrudin
MLE-24531 Debugging failing DMSDK tests
11d9f05 
Also going to move docker-compose to the root dir once this PR is ready.
@rjrudin
Trying all tests
940600e
@rjrudin
Fixing how JUnit results are captured
45eda2d
@rjrudin rjrudin force-pushed the feature/fix-tests branch from 36f9a4e to 45eda2d Compare October 8, 2025 19:01
rjrudin added 7 commits October 8, 2025 15:18
@rjrudin
Running all but DMSDK tests
73895b3
@rjrudin
Trying all test-basic tests
f10afdb
@rjrudin
Trying test-basic tests on latest-11 release
a7b4e29
@rjrudin
Back to 12-nightly
d3e5a3e 
Lots of tests failing when run against 11, understandably.

Also removed a timeout in validateDoc-test, as the timeout is being set by Jenkinsfile at the suite level.

Removed redundant lockForUpdate tests. They should pass fine, but curious to see if by removing the one that is failing, some other test randomly fails.
@rjrudin
Trying fix for validateDoc-test
424acab 
This is timing out in Jenkins, and Copilot thinks it has a fix.
@rjrudin
This should fix the test
1d938fc
@rjrudin
Adjusting geo test and skipping two others
be84796
@rjrudin rjrudin force-pushed the feature/fix-tests branch from 70da3b6 to be84796 Compare October 9, 2025 11:21
@rjrudin
Copy link
Contributor Author

rjrudin commented Oct 9, 2025

Closing, as nightly regression tests seem good.

@rjrudin rjrudin closed this Oct 9, 2025
@rjrudin rjrudin deleted the feature/fix-tests branch October 9, 2025 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anu3990 anu3990 Awaiting requested review from anu3990 anu3990 will be requested when the pull request is marked ready for review anu3990 is a code owner

@BillFarber BillFarber Awaiting requested review from BillFarber BillFarber will be requested when the pull request is marked ready for review BillFarber is a code owner

@stevebio stevebio Awaiting requested review from stevebio stevebio will be requested when the pull request is marked ready for review stevebio is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rjrudin